Skip to content

ref(opentelemetry): Vendor OTel context suppress helpers - #23030

Merged
nicohrubec merged 1 commit into
developfrom
ref/vendor-suppress-tracing
Aug 5, 2026
Merged

ref(opentelemetry): Vendor OTel context suppress helpers#23030
nicohrubec merged 1 commit into
developfrom
ref/vendor-suppress-tracing

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Aug 4, 2026

Copy link
Copy Markdown
Member

Vendors the OTel-context-based suppressTracing/isTracingSuppressed helpers from @opentelemetry/core into a local module, in preparation for dropping that dependency. The vendored copy reuses OTel's exact context key string, since third-party instrumentation like @opentelemetry/instrumentation-http writes suppression to the same key and SentryTracer reads it back to produce non-recording spans.

Fixes #22282

Vendors the OTel-context-based `suppressTracing`/`isTracingSuppressed` helpers
from `@opentelemetry/core` into a local module, so `@sentry/opentelemetry` no
longer imports them from that package. The vendored copy reuses OTel's exact
context key string, since third-party instrumentation (e.g.
`@opentelemetry/instrumentation-http`) writes suppression to the same key and
`SentryTracer` reads it back to produce non-recording spans.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec force-pushed the ref/vendor-suppress-tracing branch from cbaf7ab to ff34a3b Compare August 4, 2026 19:42
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.83 kB - -
@sentry/browser - with treeshaking flags 28.03 kB - -
@sentry/browser (incl. Tracing) 47.25 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 47.27 kB - -
@sentry/browser (incl. Tracing, Profiling) 52 kB - -
@sentry/browser (incl. Tracing, Replay) 86.59 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 76.01 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 91.28 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 103.95 kB - -
@sentry/browser (incl. Feedback) 47.13 kB - -
@sentry/browser (incl. sendFeedback) 34.67 kB - -
@sentry/browser (incl. FeedbackAsync) 39.78 kB - -
@sentry/browser (incl. Metrics) 30.91 kB - -
@sentry/browser (incl. Logs) 31.14 kB - -
@sentry/browser (incl. Metrics & Logs) 31.81 kB - -
@sentry/react 31.63 kB - -
@sentry/react (incl. Tracing) 49.51 kB - -
@sentry/vue 34.92 kB - -
@sentry/vue (incl. Tracing) 49.24 kB - -
@sentry/svelte 29.86 kB - -
CDN Bundle 31.87 kB - -
CDN Bundle (incl. Tracing) 47.54 kB - -
CDN Bundle (incl. Logs, Metrics) 33.41 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.95 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 72.75 kB - -
CDN Bundle (incl. Tracing, Replay) 85.18 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.5 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 91.01 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.31 kB - -
CDN Bundle - uncompressed 94.62 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.12 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.24 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.1 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 223.94 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.38 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.34 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.08 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.04 kB - -
@sentry/nextjs (client) 52.08 kB - -
@sentry/sveltekit (client) 47.71 kB - -
@sentry/core/server 65.52 kB - -
@sentry/core/browser 51.88 kB - -
@sentry/node 120.23 kB -0.01% -3 B 🔽
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 83.69 kB +0.01% +3 B 🔺
@sentry/aws-serverless 92.36 kB +0.01% +6 B 🔺
@sentry/cloudflare (withSentry) - minified 218.65 kB - -
@sentry/cloudflare (withSentry) 538.85 kB - -

View base workflow run

@nicohrubec
nicohrubec marked this pull request as ready for review August 5, 2026 06:02
@nicohrubec
nicohrubec requested a review from a team as a code owner August 5, 2026 06:02
@nicohrubec
nicohrubec requested review from JPeer264, andreiborza, isaacs and mydea and removed request for a team, isaacs and mydea August 5, 2026 06:02
Comment thread packages/opentelemetry/src/utils/suppressTracing.ts
@nicohrubec
nicohrubec merged commit 594f284 into develop Aug 5, 2026
393 of 395 checks passed
@nicohrubec
nicohrubec deleted the ref/vendor-suppress-tracing branch August 5, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove suppressTracing / isTracingSuppressed

2 participants